Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Add first golang written test #873

Draft
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

uncleDecart
Copy link
Collaborator

WARNING This is still WIP

I'm putting this PR here to provide visibility on what work has been done so far. Unfortunately, I couldn't split it through meaningful commits.

Signed-off-by: Pavel Abramov <[email protected]>
Since we are using config as file structure it's not reasonable to call
viper functions from openevec because that will mean that configuration
file should be present and we want to have flexibility of just passing
structure to functions

Signed-off-by: Pavel Abramov <[email protected]>
Refer to lf-edge#855

Signed-off-by: Pavel Abramov <[email protected]>
Signed-off-by: Pavel Abramov <[email protected]>
Signed-off-by: Pavel Abramov <[email protected]>
Signed-off-by: Pavel Abramov <[email protected]>
Following golang bestpractices eden CLI is moved into its
own folder under cmd folder. That way we can create other
applications using eden packages for instance integration
tests

Signed-off-by: Pavel Abramov <[email protected]>
Integration tests will have their own binary
and files used since it will all be stored in its own
cmd folder

Signed-off-by: Pavel Abramov <[email protected]>
Signed-off-by: Pavel Abramov <[email protected]>
Signed-off-by: Pavel Abramov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant